Skip to content

ref(preprod): Fix build e2e time for reruns#112527

Merged
NicoHinderling merged 1 commit intomasterfrom
04-08-ref_preprod_fix_build_e2e_time_for_reruns
Apr 8, 2026
Merged

ref(preprod): Fix build e2e time for reruns#112527
NicoHinderling merged 1 commit intomasterfrom
04-08-ref_preprod_fix_build_e2e_time_for_reruns

Conversation

@NicoHinderling
Copy link
Copy Markdown
Contributor

@NicoHinderling NicoHinderling commented Apr 8, 2026

Pass a triggered_at timestamp when dispatching the size analysis comparison task, so the preprod.size_analysis.compare.results_e2e metric measures from dispatch time instead of artifact.date_added. This fixes inflated e2e durations on re-runs where the artifact was created much earlier.

Copy link
Copy Markdown
Contributor Author

NicoHinderling commented Apr 8, 2026

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 8, 2026
@NicoHinderling NicoHinderling marked this pull request as ready for review April 8, 2026 20:53
@NicoHinderling NicoHinderling requested a review from a team as a code owner April 8, 2026 20:53
@NicoHinderling NicoHinderling merged commit d98f57b into master Apr 8, 2026
59 checks passed
@NicoHinderling NicoHinderling deleted the 04-08-ref_preprod_fix_build_e2e_time_for_reruns branch April 8, 2026 21:09
NicoHinderling added a commit that referenced this pull request Apr 8, 2026
…112528)

Follow-up to #112527. Removes the `artifact.date_added` fallback and
makes `triggered_at` a required parameter now that all producers pass
it.
george-sentry pushed a commit that referenced this pull request Apr 9, 2026
Pass a `triggered_at` timestamp when dispatching the size analysis
comparison task, so the `preprod.size_analysis.compare.results_e2e`
metric measures from dispatch time instead of `artifact.date_added`.
This fixes inflated e2e durations on re-runs where the artifact was
created much earlier.
george-sentry pushed a commit that referenced this pull request Apr 9, 2026
…112528)

Follow-up to #112527. Removes the `artifact.date_added` fallback and
makes `triggered_at` a required parameter now that all producers pass
it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants